Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create radio.csr #83

Merged
merged 1 commit into from Aug 5, 2017
Merged

Create radio.csr #83

merged 1 commit into from Aug 5, 2017

Conversation

onny
Copy link
Contributor

@onny onny commented Aug 4, 2017

@MariusBluem
Copy link
Member

Do you already have code you can share with us?

@onny
Copy link
Contributor Author

onny commented Aug 5, 2017

It's in the gitlab repository here https://git.project-insanity.org/onny/nextcloud-app-radio

@onny
Copy link
Contributor Author

onny commented Aug 5, 2017

sorry @MariusBluem my fault, now its completley public

@MariusBluem
Copy link
Member

@onny Just seen that your license value of your Info.xml is MIT license. I think only AGPL, MPL or Apache is supported by the appstore as value ...

See: https://nextcloudappstore.readthedocs.io/en/latest/developer.html

@BernhardPosselt
Copy link
Member

There was a change recently which adds MIT, not sure if it's deployed already though

@MariusBluem
Copy link
Member

👍

@onny
Copy link
Contributor Author

onny commented Aug 5, 2017

So should I change the license or could I stick with MIT?

@LukasReschke LukasReschke merged commit ce3ce98 into nextcloud:master Aug 5, 2017
@LukasReschke
Copy link
Member

Certificate at https://github.com/nextcloud/app-certificate-requests/blob/6090a6dae470847fb747b547b3a3aa0663664b2e/radio/radio.crt :)

About the license, I'd suggest just try if the current deployed appstore version supports MIT, if not change the license or wait until the app store has been updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants